Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enable role and profile caching #2533

Merged
merged 7 commits into from
May 31, 2024
Merged

Conversation

fmauNeko
Copy link
Contributor

What does this PR do ?

Enable caching of roles and profiles in the security module.

@fmauNeko fmauNeko force-pushed the feature/cache-profiles-roles branch from eccd70b to 571dccf Compare May 28, 2024 15:44
The internal memory cache has been removed, and the cache testing is handled by the ObjectRepository
tests
@fmauNeko fmauNeko marked this pull request as ready for review May 29, 2024 15:47
Serializing a Map is apparently a bad idea
@fmauNeko fmauNeko force-pushed the feature/cache-profiles-roles branch from bd07ec2 to 9cf9d69 Compare May 30, 2024 07:48
Copy link

sonarcloud bot commented May 30, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

test/core/security/profileRepository.test.js Outdated Show resolved Hide resolved
@alexandrebouthinon alexandrebouthinon merged commit e87f5de into 2-dev May 31, 2024
34 checks passed
@alexandrebouthinon alexandrebouthinon deleted the feature/cache-profiles-roles branch May 31, 2024 09:47
@kuzzle
Copy link
Contributor

kuzzle commented Jun 3, 2024

🎉 This PR is included in version 2.30.1-beta.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

@kuzzle
Copy link
Contributor

kuzzle commented Jul 22, 2024

🎉 This PR is included in version 2.31.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@kuzzle kuzzle added the released This issue/pull request has been released. label Jul 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
released on @beta released This issue/pull request has been released.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants